a11y: Remove idle handler for switch action handling
authorBenjamin Otte <otte@redhat.com>
Tue, 5 Jul 2011 12:39:12 +0000 (14:39 +0200)
committerBenjamin Otte <otte@redhat.com>
Wed, 6 Jul 2011 14:40:04 +0000 (16:40 +0200)
gtk/a11y/gtkswitchaccessible.c
gtk/a11y/gtkswitchaccessible.h

index 71e9e913dfe56dcd529be675bf603428d9af0110..2ca045185b21c031b2771a621f8f8d7806aa208e 100644 (file)
@@ -55,17 +55,6 @@ gtk_switch_accessible_ref_state_set (AtkObject *accessible)
   return state_set;
 }
 
-static void
-gtk_switch_accessible_finalize (GObject *obj)
-{
-  GtkSwitchAccessible *accessible = (GtkSwitchAccessible *)obj;
-
-  if (accessible->action_idle)
-    g_source_remove (accessible->action_idle);
-
-  G_OBJECT_CLASS (gtk_switch_accessible_parent_class)->finalize (obj);
-}
-
 static void
 gtk_switch_accessible_initialize (AtkObject *accessible,
                                   gpointer   widget)
@@ -80,11 +69,8 @@ gtk_switch_accessible_initialize (AtkObject *accessible,
 static void
 gtk_switch_accessible_class_init (GtkSwitchAccessibleClass *klass)
 {
-  GObjectClass *object_class = G_OBJECT_CLASS (klass);
   AtkObjectClass *atk_class = ATK_OBJECT_CLASS (klass);
 
-  object_class->finalize = gtk_switch_accessible_finalize;
-
   atk_class->initialize = gtk_switch_accessible_initialize;
   atk_class->ref_state_set = gtk_switch_accessible_ref_state_set;
 }
@@ -92,7 +78,6 @@ gtk_switch_accessible_class_init (GtkSwitchAccessibleClass *klass)
 static void
 gtk_switch_accessible_init (GtkSwitchAccessible *self)
 {
-  self->action_idle = 0;
 }
 
 static gint
@@ -111,32 +96,11 @@ gtk_switch_action_get_name (AtkAction *action,
   return "toggle";
 }
 
-static gboolean
-idle_do_action (gpointer data)
-{
-  GtkSwitchAccessible *accessible = data;
-  GtkWidget *widget;
-  GtkSwitch *sw;
-
-  widget = gtk_accessible_get_widget (GTK_ACCESSIBLE (data));
-  sw = GTK_SWITCH (widget);
-
-  accessible->action_idle = 0;
-
-  if (widget == NULL ||
-      !gtk_widget_is_sensitive (widget) || !gtk_widget_get_visible (widget))
-    return FALSE;
-
-  gtk_switch_set_active (sw, !gtk_switch_get_active (sw));
-
-  return FALSE;
-}
-
 static gboolean
 gtk_switch_action_do_action (AtkAction *action,
                              gint       i)
 {
-  GtkSwitchAccessible *accessible;
+  GtkSwitch *sw;
   GtkWidget *widget;
 
   widget = gtk_accessible_get_widget (GTK_ACCESSIBLE (action));
@@ -149,10 +113,8 @@ gtk_switch_action_do_action (AtkAction *action,
   if (!gtk_widget_is_sensitive (widget) || !gtk_widget_get_visible (widget))
     return FALSE;
 
-  accessible = (GtkSwitchAccessible *)action;
-
-  if (!accessible->action_idle)
-    accessible->action_idle = gdk_threads_add_idle (idle_do_action, accessible);
+  sw = GTK_SWITCH (widget);
+  gtk_switch_set_active (sw, !gtk_switch_get_active (sw));
 
   return TRUE;
 }
index 694a83cb334fca615f82aefc392c78975f66014c..3cb66a98a1cbac83c750aed01da6f96e5d060592 100644 (file)
@@ -37,8 +37,6 @@ typedef struct _GtkSwitchAccessibleClass GtkSwitchAccessibleClass;
 struct _GtkSwitchAccessible
 {
   GtkWidgetAccessible parent;
-
-  guint action_idle;
 };
 
 struct _GtkSwitchAccessibleClass